window: Defer focus setting until after paint

Commit 3dbf5038fab8eb0d8c11 tried to defer focus changes
until after rendering is done. But it failed to do so, since
the toplevel ::render handler is still before rendering of
popups that are attached to that toplevel. To do this
properly, we need to do it in the AFTER_PAINT frame clock
phase.

Fixes: #3725

Merge branch 'no-scroll-cursor' into 'master'

scrolledwindow: Stop using scroll cursors

See merge request GNOME/gtk!3416

Merge branch 'ebassi/ci-docs' into 'master'

ci: Drop the split Fedora images

See merge request GNOME/gtk!3415

scrolledwindow: Stop using scroll cursors

We used to override cursor to use all-scroll while the
content is being scrolled. Unfortunately, there is several
problems with this:
- It is really only expected certain devices, and we don't
  have the device information on Wayland
- With the way cursor setting works in GTK4, non-NULL cursors
  of the content (eg the text views ibeam) win, making the
  scroll cursor not show up
- Under X11, we seem to miss scroll end events and then
  the scroll cursor gets stuck
Therefore, just remove this feature.

ci: Drop the split Fedora images

Now that we don't need the whole of Pandoc to build our documentation,
we can go back to a single Fedora image for our CI.

Merge branch 'half-float' into 'master'

ngl: Use fp16 for colors

See merge request GNOME/gtk!3405

Merge branch 'font-fallback-fix' into 'master'

wayland: Improve font setting fallback more

Closes #3838

See merge request GNOME/gtk!3412

Work around compiler shortcomings on macOS

alias attributes don't work on Darwin, so
do without.

gsk: Add runtime checks for F16C

Use an IFUNC resolver to determine whether we can use
intrinsics for FP16 conversion. This requires the functions
to be no longer inline.

Sadly, it turns out that __builtin_cpu_supports ("f16c")
doesn't compile on the systems where we want it to prevent
us from getting a SIGILL at runtime.

Add an option to disable F16C

And add a compile time check for the presence of this
CPU feature.

Move color conversion up

Doing the color conversion early lets us reuse
the results whenever we do more than one draw
per node.

ngl: Use a constant for (fp16)-1

No need to convert that again for every glyph.

ngl: Reduce vertex storage

We only have one shader that uses the color2 attribute,
and it doesn't use the uv attribute, so save vertex
memory by putting those in the same space.

This reduce the per vertex space from 32 to 24 bytes.

ngl: Use fp16 for colors

This reduces the size of our Vertex struct from
48 to 32 bytes. It would be nicer if we could store
the colors in fp16 format in the rendernodes, and
avoid conversion here. But this is still good.

wayland: Improve font setting fallback more

We may get a response from the portal that contains
no useful settings at all. In that case, we should
fallback as well.

Fixes: #3838

Merge branch 'wip/carlosg/stuck-frames' into 'master'

surface: Always set PHASE_PAINT as pending when updates are scheduled

Closes #3750

See merge request GNOME/gtk!3411

surface: Always set PHASE_PAINT as pending when updates are scheduled

At times (most often when closing subsurfaces that are scheduling
relayouts) the PHASE_PAINT handling gets broken with the following
sequence:

1. Surface receives wl_callback.done for the previous frame.
   Surface is thawed.
2. A new update on the surface is scheduled. PHASE_PAINT is
   requested directly on the frame clock. priv->pending_phase is
   left unset in the surface.
3. Surface gets frozen
4. Frame clock processes the update scheduled at 2. The surface
   is frozen, so paint is prevented. PHASE_PAINT is considered
   handled.
5. Compositor emits wl_callback.done again. Surface is thawed.
6. At this point the machinery is off
   - The surface didn't paint but has pending update regions
   - priv->draw_needed is set in the toplevel and other portions
     of the widget tree
   - So queueing redraws is ineffective at eventually calling
     gdk_surface_schedule_update() again on the toplevel surface.
   - We don't paint anymore, so this broken state is not flushed
     until other subsurface changes manage to schedule the missing
     update.

To fix this, always set PHASE_PAINT in priv->pending_phase when
doing gdk_surface_schedule_update(). If the frame clock turns
around before the surface is thawed, it will still be waiting to
be processed the next iteration.

Fixes: https://gitlab.gnome.org/GNOME/gtk/-/issues/3750

Merge branch 'font-settings-fallback-4' into 'master'

wayland: Improve font setting fallback

See merge request GNOME/gtk!3407

wayland: Improve font setting fallback

When we don't get stettings from the portal, the current
fallback is 'awful fonts'. There is no need for that. Instead,
set the fallback values to grayscale antialiasing with slight
hinting.

Merge branch 'textview-last-line' into 'master'

textview: Fix a problem with the last line

Closes #3835

See merge request GNOME/gtk!3403

textview: Fix a problem with the last line

Sometimes, we missed the last line for display.

Fixes: #3835

Merge branch 'matthiasc/for-master' into 'master'

window: Actually enable the inspector by default

Closes #3831

See merge request GNOME/gtk!3401

modelbutton: Avoid a crash

We may not have a popover menu ancestor. The check for
this was forgotten in one of the branches here.

Fixes: #3831

window: Actually enable the inspector by default

427d216081ea0fd9e2045e changed the default in the schema,
but didn't handle cases where the schema isn't found.

Merge branch 'matthiasc/for-master' into 'master'

textlayout: Remove some unnecessary code

See merge request GNOME/gtk!3400

ngl: Fix an oversight

We were special-casing 2D affine transforms,
but overlooked even simpler transforms.

textlayout: Remove an unused function

textlayout: Avoid allocations for line iteration

No need to collect the lines firs, and allocate memory
for that. We can just iterate over them right away.

textlayout: Introduce some locals

No need to call _gtk_text_buffer_get_btree over and over.

textlayout: Create the cursor snapshot on demand

We don't need it all the time.

textattributes: Small docs fix

textbtree: Return tags in a GPtrArray

One of the callers prefers that, it lets us avoid
copying the array in one place, and generally makes
for better code.

textlayout: Only get selection color once

No need to do this for every paragraph that is
inside the selection.

texttag: Just use qsort

No point in trying to optimize this.

textbtree: Inline node data handling

textbtree: Speed up _gtk_text_btree_find_line_top

Another place where we can avoid allocating a stack
piecemeal.

textbtree: Don't opencode realloc

We can just use g_realloc here.

textbtree: Speed up _gtk_text_line_char_index

No need to allocate a stack piecemeal here.

textbtree: Avoid malloc in one place

textlayout: Don't call render_para too much

render_para produces no nodes for an empty paragraph.
Don't call it over and over for those.

textlayout: Remove some unnecessary code

The same field is set again 2 lines down.

Update Chinese (Taiwan) translation

(cherry picked from commit 3d3912d7f4ea27ddb9aa79f02bce6bc2d4889bb0)

Merge branch 'matthiasc/for-master' into 'master'

gsk: Rename resource paths

See merge request GNOME/gtk!3399

Merge branch 'dotted-lines3' into 'master'

Modernize error underlines

See merge request GNOME/gtk!3398

Modernize error underlines

Implement PANGO_UNDERLINE_ERROR as a dotted line, instead of
squiggly, which looks quite dated.

gsk: Move shader resources

Move the resources of each renderer to its subdirectory.
We've previously done that for the ngl renderer, but it
is better to be consistent and do it for all the renderers.

gsk: Rename resource paths

Make the shaders of the gl renderer live under
/org/gtk/libgsk/gl. This is purely cosmetic.

Merge branch 'matthiasc/for-master' into 'master'

node test: Improve --generate

See merge request GNOME/gtk!3397

settings: Make font-size changes apply immediately

We need to invalidate the style when font-size changes,
because we propagate this value through the initial
value of the CSS font-size property, and it will not
be recomputed otherwise.

immulticontext: Unset client widget on delegate change

Forgetting to do so was causing the Wayland im context
to leave behind a dead event controller. This was showing
up as a crash when closing the inspector after changing
the im-module property of a GtkText widget. The crash
was delayed until closing the inspector because the
inspector keeps a ref on the event controllers of the
currently shown widget.

imwayland: Name event controllers

This helps with debugging.

widget-factory: Name an event controller

This helps with debugging.

text: Name all event controllers

This helps with debugging.

inspector: Support copying nodes to clipboard

Add a button that copies the serialization of the
selected node to the clipboard.

node-editor: Support pasting the image

Add a button that copies the image to the clipboard.

nodeparser: Change the serialization

Arrange things so that non-child parameters
are always printed before the children. This
greatly helps with readability, which really
suffers when there's hundreds of lines of indented
children between the node start and its parameters.

Update all affected tests.

node test: Improve --generate

This is a pet peeve of mine: When we call
g_test_init() before handling --generate,
the random seed spew pollutes the output.
Highly annoying. I've fixes many test binaries
over the years, but more keep popping up.

Merge branch 'fix-downscaled-textures' into 'master'

ngl: Fix downscaled textures

See merge request GNOME/gtk!3395

Merge branch 'matthiasc/for-master' into 'master'

comboboxtext: Remove misleading docs

Closes #3824

See merge request GNOME/gtk!3396

Merge branch 'fix-initial-font-selection' into 'master'

fontchooser: Fix initial font selection

Closes #3687

See merge request GNOME/gtk!3392

comboboxtext: Remove misleading docs

The entry is no longer accessible.
The docs were outdated.

Fixes: #3824

ngl: Avoid huge intermediate textures

Instead of rendering the unclipped child to a texture
(and risking blowing the texture size limit, and bad
downscaling), just render the clipped region, and live
with the fact that we can't cache the rendered texture.

This avoid bad artifacts when scrolling long textviews
in rounded clips.

ngl: Plug a memory leak

This was introduced in f9457af128012fd3935a69fdcefb.

ngl: Fix downscaled textures

It is not pretty, but at least it works now.

Merge branch 'flat-button-activating' into 'master'

button: Style .keyboard-activating on frameless buttons

See merge request GNOME/gtk!3394

button: Style .keyboard-activating on frameless buttons

The use of the keyboard-activating CSS class for buttons was added
in [1], but the style did not apply to buttons with has-frame=FALSE.

[1] 00923615f4 ("button: Add back visual feedback for keynav", 2021-04-01)

Update Dutch translation

(cherry picked from commit a356bf6fac604e37cfd4ac6e9c2486a3453ac181)

fontchooser: Fix initial font selection

The change in 740559a54f to populate the list incrementally
broke initial font selection. Fix that, by trying to select
until the incremental filling is done.

Fixes: #3687

Merge branch 'theme-reorg' into 'master'

Reorganize our themes

See merge request GNOME/gtk!3079

Make theme fallback working with the new names

Since we are likely going to see theme names like
Adwaita and HighContrast, make fallback work as follows:

Adwaita -> Default
Adwaita:dark -> Default:dark
HighContrast -> Default:hc
HighContrast:dark -> Default:hc-dark
HighContrastInverse -> Default:hc-dark

Other themes will fall back to Default, as before.

Merge branch 'matthiasc/for-master' into 'master'

testsuite: Sort gsk compare tests

See merge request GNOME/gtk!3389

testsuite: Sort gsk compare tests

Easier to find things that way.

Add some more gsk tests

The css-background test contains a node pattern that
is optimized in the ngl renderer.

Add one more compose test

Merge branch 'button-activate-feedback' into 'master'

button: Add back visual feedback for keynav

Closes #3813

See merge request GNOME/gtk!3390

button: Add back visual feedback for keynav

We lost the visual feedback for activating a button
via Space or Enter when the :active pseudo-state became
managed. Bring it back with a style class.

Fixes: #3813

Merge branch 'matthiasc/for-master' into 'master'

Add one more compose test

See merge request GNOME/gtk!3388

Add one more compose test

Merge branch 'us-intl-compose' into 'master'

Revert Compose sequence changes

Closes #3807

See merge request GNOME/gtk!3386

Add more compose tests

Test the sequences whose demise made people
unhappy.

Revert Compose sequence changes

This was breaking muscle memory of people with
the us intl keyboard layout, for important keys
such as '. The unfortunate side-effect is that
our handling of <dead_acute> is a bit hampered
by sequences that don't fit the pattern. But
such is life.

Fixes: #3807

Merge branch 'builder-tag' into 'master'

docs: Escape tags in GtkBuilder description

See merge request GNOME/gtk!3385

docs: Escape tags in GtkBuilder description

Fixes the tags and the rest of the gi-docgen created HTML page not being
visible in a browser.

Drop Adwaita and HighContrast themes

These themes will reappear in libadwaita in due course.

widget-factory: Use the new themes

Update the Style menu to switch between the variants
of the Default theme.

Change the default theme to "Default"

We are not shipping Adwaita anymore.

Make the theme variants available as themes

This makes it easier to implement the theme switching
in widget-factory, since we don't have a variant setting.

Reorganize our themes

Rename the included theme to Default, with 4 variants:
light, dark, hc, hc-dark. This replaces Adwaita,
Adwaita:dark, HighContrast and HighContrastInverse.
We still make the themes available under these names,
and we still set up Adwaita-dark and HighContrastInverse
as the dark variants of Adwaita and HighContrast.

The unification of the theme variants under Default
is not quite perfect; it would be nice to merge
the assets/ and assets-hc/ subdirectories and render
all assets from a single svg file.

Merge branch 'wip/otte/for-master' into 'master'

snapshot: scale border widths when appending border nodes

Closes #3819

See merge request GNOME/gtk!3384

snapshot: Move sanity checks to a better place

Fixes cases where the sanity checks would reject valid code, like when
using GL shaders on a scaled transform.

Fixes #3819

snapshot: scale border widths when appending border nodes

Found by Matthias on IRC while arguing about GtkSnapshot being too
complicated.

Merge branch 'filechooser-signal-disconnect' into 'master'

filechooserwidget: Disconnect settings signal on dispose

See merge request GNOME/gtk!3383

Merge branch 'wip/exalm/overshoot-changes' into 'master'

scrolledwindow: Cancel overshoot on dimension changes

Closes #3752

See merge request GNOME/gtk!3300

scrolledwindow: Cancel overshoot on dimension changes

If we scroll down in a list that's still being filled, we hit the edge and
initiate overshoot, and then the adjustment's upper value increases. This
leads to an unwanted bounce back.

Additionally, if in a similar situation the upper value decreases, the
overscroll glow gets stuck.

Update kinetic scrolling upper and lower value on changes, and immediately
cancel it if dimensions on that side change.

Fixes https://gitlab.gnome.org/GNOME/gtk/-/issues/3752

filechooserwidget: Reduce code duplication for disconnecting settings signal

Instead of getting current display before calling settings signal removal,
do it inside remove function and only if there is a signal connection to remove.

filechooserwidget: Disconnect settings signal on dispose

Settings signal was not being disconnected on dispose causing
application crash when user changed icon theme after dialog disposal.

Merge branch 'ebassi/broadway-docs' into 'master'

docs: Update the Broadway instructions

See merge request GNOME/gtk!3380

Merge branch 'ebassi/docs-links' into 'master'

docs: Update the links to the docs of GTK dependencies

Closes #3817

See merge request GNOME/gtk!3381

Merge branch 'offscreen-fixes' into 'master'

Offscreen fixes

See merge request GNOME/gtk!3379

gsk: Add another test

Compare clipped repeat nodes. Must skip cairo here
since it blurred the child by scaling after rendering.

Also skip the gl renderer, since it hasn't been fixed
for this yet. ngl passes this test.